Rename debug target to dev in build output

This renames the debug target to dev in the build output for non-release
builds.  This includes the output of commands such as `cargo build`,
`cargo test`, and `cargo doc`.

Closes #3519

Auto merge of #3524 - sdroege:test-build-doc-test-check-status, r=alexcrichton

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

As suggested by @alexcrichton in https://github.com/rust-lang/cargo/pull/3515#discussion_r95253461

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

Auto merge of #3489 - matklad:rel-ws, r=alexcrichton

Use canonical paths for checking equality

Auto merge of #3515 - sdroege:doc-all, r=alexcrichton

 Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3518 - Manishearth:static-mut, r=alexcrichton

Replace static mut bool with atomicbool

unsafe in tests is fine, but it can be trivially avoided in this case.

r? @alexcrichton

cc @froydnj

Replace static mut bool with atomicbool

Collect library/binary names when documenting even if a spec was given

This seems more consistent and generally should work better.

Check if the "doc --all" tests exit with exit code 0

Auto merge of #3511 - sdroege:build-all, r=alexcrichton

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3510 - froydnj:cross-compilation-test-checking, r=alexcrichton

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an appropriate cross standard library is not found.  This behavior is unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an appropriate cross-compilation setup is available.  Once that check has been done, a single test can fail with an appropriate `panic!` message while the other tests silently pass.  The `panic!` message can inform the user about their options, either setting an appropriate environment variable, or using rustup to install the necessary cross standard library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

The code is admittedly gnarly, with synchronization and `unsafe` sections, but I wasn't sure how to make it any better. Suggestions welcome!

Fixes #3086.

Add tests for "build --all"

These are basically the same as the ones from "test --all"

Add tests for "doc --all"

These are basically the same as the ones from "test --all" and "build --all"

Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Normalize paths before checking for equality

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an
appropriate cross standard library is not found.  This behavior is
unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an
appropriate cross-compilation setup is available.  Once that check has
been done, a single test can fail with an appropriate `panic!` message
while the other tests silently pass.  The `panic!` message can inform
the user about their options, either setting an appropriate environment
variable, or using rustup to install the necessary cross standard
library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu

You may need to install runtime libraries for your Linux distribution as well.
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

Fixes #3086.

Auto merge of #3509 - froydnj:colored-shell-tests, r=alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3508 - petrochenkov:bsover, r=alexcrichton

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

Fixes https://github.com/rust-lang/cargo/pull/3438#discussion_r94018355
r? @alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Auto merge of #3504 - froydnj:win32-dependencies, r=alexcrichton

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only required under Windows.

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only
required under Windows.

Auto merge of #3427 - nrc:api-check-2, r=alexcrichton

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

r? @alexcrichton

Auto merge of #3505 - froydnj:read2-delete-try, r=alexcrichton

delete unused try macro from read2.rs

AFAICS, this is unused.

Auto merge of #3507 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Add `--profile check` to `cargo rustc`

delete unused try macro from read2.rs

Auto merge of #3495 - sfackler:bump-shell-escape, r=alexcrichton

Update shell-escape

r? @alexcrichton

Auto merge of #3500 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Try using before_deploy on AppVeyor

Try using before_deploy on AppVeyor

Update shell-escape

review changes, round 2 part b

Remove the ContinueBuild concept, be lenient about missing files in all cases, and undo the fingerprint changes (not an issue now we're not stopping the build early).

review changes, round 2

Mostly focussing on the ergonomics of the API - removes with_* methods with closures, and replaces generics with trait objects.

Review changes

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

Auto merge of #3468 - nrc:metadata-emit, r=alexcrichton

cargo check: use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571 (don't land before that does)

r? @alexcrichton

Fixup tests

Use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571

Auto merge of #3485 - sfackler:useragent, r=alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

r? @alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

Auto merge of #3480 - alexcrichton:one-flaky-test, r=brson

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Auto merge of #3477 - sfackler:shell-escape, r=alexcrichton

Use externally sourced shell-escape

Closes #3374

r? @alexcrichton

Use externally sourced shell-escape

Closes #3374

Auto merge of #3470 - alexcrichton:no-more-urls, r=steveklabnik

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Auto merge of #3435 - alexcrichton:bump-libssh2, r=brson

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Auto merge of #3417 - alexcrichton:fix-svn, r=brson

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

Auto merge of #3412 - alexcrichton:fix-recursion, r=brson

Fix recursion in build_path_deps

We were slightly too eager to follow pointers!

Closes #3330

Auto merge of #3467 - alexcrichton:verbose, r=alexcrichton

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Auto merge of #3438 - petrochenkov:bsdoc, r=alexcrichton

Add some more details to build script docs

r? @alexcrichton

Add some more details to build script docs

Auto merge of #3458 - alexcrichton:docs, r=steveklabnik

Update README build instructions

Closes #3437

Auto merge of #3454 - alexcrichton:bump1, r=alexcrichton

Bump crates-io to 0.6.0

Auto merge of #3439 - gentoo90:bashcomp, r=alexcrichton

Bashcomp

Update bash completion

Auto merge of #3426 - nrc:check-fix-2, r=alexcrichton

Test for #3419

Based on top of https://github.com/rust-lang/cargo/pull/3425.

I'm not sure if this is an acceptable test because it downloads rustc-serialize.

r? @alexcrichton

Auto merge of #3436 - matklad:abs-target, r=alexcrichton

Make sure targets always use absolute path

Closes #3416

I'd want to add a test for it later. Seems a bit suspicious that this change has not caused any test to fail.

Update README build instructions

Auto merge of #3457 - alexcrichton:no-check-on-beta, r=alexcrichton

Only run check tests on nightly

Only run check tests on nightly

Bump crates-io to 0.6.0

Test for cargo check with macro and both lib and bin.

Closes #3419

Add a test

Includes adding facility for checking that output does not contain a string.

Update bash completion

Unify indentation in bash completion

Make sure targets always use absolute path

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

Auto merge of #3430 - alexcrichton:bump, r=brson

Bump to 0.17.0

Bump to 0.17.0

Auto merge of #3424 - matklad:naming-things, r=alexcrichton

Minor, more explicit names

Ultraminor renaming. Have to look at this tests because it fails if you link with older libgit.

Auto merge of #3413 - matklad:ref, r=alexcrichton

Refactor resolve_dependencies

This moves `resolve_dependencies` function from `compile.rs` into the `resolve.rs`, where it more reasonably belongs.

Auto merge of #3410 - fmdkdd:fix-3390-message-format-parse-stderr, r=alexcrichton

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

make sure we use the check profile where possible for lib deps.

Fixes #3418

Minor, more explicit names

Don't carry source ids along with sources

Be more explicit about temporary workspaces

Add source_id method to the Source

Fix recursion in build_path_deps

We were slightly too eager to follow pointers!

Closes #3330

Add a test for `--message-format` parsing the mixed output of rustc

Rename resolve_dependencies -> resolve_ws_precisely

Simplify `resolve_ws` API

Move `resolve_dependencies` into resolve.rs

Auto merge of #3411 - alexcrichton:tweaks, r=alexcrichton

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Minor build tweaks

* Clean out OpenSSL when building
* Test 32-bit Cargo on OSX, not the 64-bit version on the 32-bit builder

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

Auto merge of #3409 - matklad:workspace-manifest-error, r=alexcrichton

Don't ignore errors in workspace manifest

Closes #3407

If we find `Cargo.toml` during an up tree traversal, we'd better not to ignore errors within it!

Don't ignore errors in workspace manifest

Auto merge of #3393 - alexcrichton:fix-libz-linkage, r=brson

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Closes #3384

Auto merge of #3403 - matklad:doc-loc, r=alexcrichton

cargo doc: show where are docs

Let's print the file we are trying to open. It's useful when we fail to detect the browser, because the user would be able to copy-paste url manually.

Bump libz-sys to 1.0.9

This just switches libz to always link statically instead of relying on the
system zlib. For MinGW it seems that linkage may default pull in a DLL, which is
almost never what we want.

Also update curl-sys to fix a build issue on MinGW.

Closes #3384

Auto merge of #3402 - matklad:packages, r=alexcrichton

Refactor packages specification

A small refactoring to avoid returning a triple.

cargo doc: show where are docs

Refactor packages specification

Auto merge of #3396 - pornel:test, r=alexcrichton

Test for issue #2064

I've tried to reproduce the problem from #2064, but to my surprise the test passes reliably :)

Auto merge of #3296 - nrc:check, r=alexcrichton

cargo check

~~This is not finished - the big omission is no tests, and it needs some more testing too. It also requires https://github.com/rust-lang/rust/pull/37681.~~

However, this is my first non-trivial Cargo patch, so I'd like to get some early feedback.

r? @alexcrichton

and cc @rust-lang/tools since this adds a new Cargo sub-command (although we have previously discussed and approved the idea in a tools meeting).

Auto merge of #3398 - alexcrichton:git-rev, r=alexcrichton

Fix git rev showing up in version number

Makefiles didn't support an out-of-tree build, so needed to update them to do
so.